Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

You cannot drop a column ref'd in a partial index #16413

Merged
merged 1 commit into from
Mar 15, 2023
Merged

You cannot drop a column ref'd in a partial index #16413

merged 1 commit into from
Mar 15, 2023

Conversation

rmloveland
Copy link
Contributor

Fixes DOC-7050

Relates to DOC-7029

@vercel
Copy link

vercel bot commented Mar 3, 2023 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
docs ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 3, 2023 at 8:14PM (UTC)

@rmloveland rmloveland requested a review from mgartner March 3, 2023 20:03
@github-actions
Copy link

github-actions bot commented Mar 3, 2023

Files changed:

@rmloveland rmloveland mentioned this pull request Mar 3, 2023
Merged
@netlify
Copy link

netlify bot commented Mar 3, 2023

✅ Netlify Preview

Name Link
🔨 Latest commit 3d84959
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-docs/deploys/6402528b8a77c400083d7bc7
😎 Deploy Preview https://deploy-preview-16413--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@cloudflare-workers-and-pages
Copy link

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3d84959
Status: ✅  Deploy successful!
Preview URL: https://f067d10b.docs-7ke.pages.dev
Branch Preview URL: https://20230303-doc-7050.docs-7ke.pages.dev

View logs

@rmloveland
Copy link
Contributor Author

Hi @mgartner ! Mind taking a look at this docs PR? it attempts to update the docs based on the limitation described in TA-96924, which has been published

Entirely possible there is nuance here that this docs update does not capture, happy to update the PR as needed!

mgartner
mgartner approved these changes Mar 13, 2023
View reviewed changes
Copy link
Contributor

@mgartner mgartner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@rmloveland rmloveland requested a review from ianjevans March 13, 2023 20:45
@rmloveland rmloveland merged commit e49bbbf into master Mar 15, 2023
@rmloveland rmloveland deleted the 20230303-DOC-7050 branch March 15, 2023 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ianjevans ianjevans ianjevans approved these changes

@mgartner mgartner mgartner approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rmloveland @mgartner @ianjevans